Skip to content

[SPARK-46953][TEST]] Wrap withTable for a test in ResolveDefaultColumnsSuite#44993

Closed
yaooqinn wants to merge 1 commit intoapache:masterfrom
yaooqinn:SPARK-43742
Closed

[SPARK-46953][TEST]] Wrap withTable for a test in ResolveDefaultColumnsSuite#44993
yaooqinn wants to merge 1 commit intoapache:masterfrom
yaooqinn:SPARK-43742

Conversation

@yaooqinn
Copy link
Member

@yaooqinn yaooqinn commented Feb 2, 2024

What changes were proposed in this pull request?

The table is not cleaned up after this test; test retries or upcoming new tests reused 't' as the table name will fail with TAEE.

Why are the changes needed?

fix tests as FOLLOWUP of SPARK-43742

Does this PR introduce any user-facing change?

no

How was this patch tested?

this test itself

Was this patch authored or co-authored using generative AI tooling?

no

@yaooqinn yaooqinn requested a review from cloud-fan February 2, 2024 05:39
@github-actions github-actions bot added the SQL label Feb 2, 2024
@yaooqinn yaooqinn changed the title [SPARK-43742][TEST] Wrap withTable for a test in ResolveDefaultColumnsSuite [SPARK-43742][TEST][FOLLLOWUP] Wrap withTable for a test in ResolveDefaultColumnsSuite Feb 2, 2024
@yaooqinn yaooqinn changed the title [SPARK-43742][TEST][FOLLLOWUP] Wrap withTable for a test in ResolveDefaultColumnsSuite [SPARK-46953][TEST]] Wrap withTable for a test in ResolveDefaultColumnsSuite Feb 2, 2024
@yaooqinn
Copy link
Member Author

yaooqinn commented Feb 2, 2024

Thanks! Merged to master/3.5.

@yaooqinn yaooqinn closed this in a343242 Feb 2, 2024
yaooqinn added a commit that referenced this pull request Feb 2, 2024
…sSuite

### What changes were proposed in this pull request?

The table is not cleaned up after this test; test retries or upcoming new tests reused 't' as the table name will fail with TAEE.

### Why are the changes needed?

fix tests as FOLLOWUP of SPARK-43742

### Does this PR introduce _any_ user-facing change?

no
### How was this patch tested?

this test itself
### Was this patch authored or co-authored using generative AI tooling?

no

Closes #44993 from yaooqinn/SPARK-43742.

Authored-by: Kent Yao <yao@apache.org>
Signed-off-by: Kent Yao <yao@apache.org>
(cherry picked from commit a343242)
Signed-off-by: Kent Yao <yao@apache.org>
@yaooqinn yaooqinn deleted the SPARK-43742 branch February 2, 2024 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants